Skip to content

Fixing documentation issue with core install command #206

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from
Closed

Fixing documentation issue with core install command #206

wants to merge 1 commit into from

Conversation

jeff-winn
Copy link
Contributor

Resolves issue #205 wherein the command documentation shows an = instead of an @ symbol as expected during execution.

@CLAassistant
Copy link

CLAassistant commented May 10, 2019

CLA assistant check
All committers have signed the CLA.

@masci
Copy link
Contributor

masci commented Jun 7, 2019

Hi @jeff-winn sorry for the late reply, this fell through the cracks.

Function was moved under the cli package, do you think you can move the patch there? That would be great!

@masci masci self-assigned this Jun 7, 2019
@jeff-winn
Copy link
Contributor Author

Not sure I follow what you’re asking of me, could you elaborate?

Thanks!

@jeff-winn
Copy link
Contributor Author

Disregard, I found what you were talking about. I’ll redo it and submit another pull request.

@jeff-winn jeff-winn closed this Jun 8, 2019
per1234 added a commit that referenced this pull request Aug 9, 2021
Make symlink prohibition rule catch links with folder target
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants